Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP - upgrade to webpack 5 #11326

Closed
wants to merge 1 commit into from
Closed

WIP - upgrade to webpack 5 #11326

wants to merge 1 commit into from

Conversation

ndelangen
Copy link
Member

@ndelangen ndelangen commented Jun 26, 2020

Issue: #9216

@gaetanmaisse
Copy link
Member

@ndelangen I will have a few commits to add to this PR to clean things related to Yarn PnP so can you ping me when you think it will be ok for me to do these changes?

@kirill-konshin
Copy link

Is there any way I can help to speed up this merge? We've migrated to Webpack 5 and only thing that is not migrated is Storybook...

@shilman
Copy link
Member

shilman commented Jul 22, 2020

@kirill-konshin Don't hold your breath. We're in RC for 6.0 and will re-evaluate this in August after that ships. Since this is a breaking change, the earliest we can merge release as is would be 7.0. However, to support early webpack5 users like yourself and support the evolution of webpack, we could potentially publish an OFFICIAL storybook fork. If you have thoughts on how that could work, that might help us push this along after 6.0 is out.

@kirill-konshin
Copy link

@shilman thank you for clarification. I'd be happy to be among first adopters, alpha/beta does not matter. Probably a dedicated dist-tag would work, just like always.

@ndelangen
Copy link
Member Author

We'd be very welcoming to any help you could offer us @kirill-konshin 🙇

ethanwu10 added a commit to redpwn/rctf that referenced this pull request Aug 9, 2020
Roll back to Webpack 4 because Storybook does not support Webpack 5 for
the near future (see
storybookjs/storybook#11326 (comment)).

Use @storybook/react (which includes React as a dependency) instead of
Preact because the Storybook build breaks when using theme-ui (most
likely because of the bundled React dependency).
@kirill-konshin
Copy link

We'd be very welcoming to any help you could offer us @kirill-konshin 🙇

So is there a dist tag that I can take and try?

I only see

% npm info @storybook/react

dist-tags:
latest: 6.0.5      next: 6.0.0-rc.30  

How can I install this branch into my project?

@kirill-konshin
Copy link

@shilman wrote:

@kirill-konshin https://github.com/storybookjs/storybook/blob/next/CONTRIBUTING.md#development-guide

Unfortunately yarn link won't work because we have a large team of 40+ engineers and we need to have this running on CI. So dist-tag is a must here... I can clone and publish under my own scope, but that's an overkill...

@shilman
Copy link
Member

shilman commented Aug 13, 2020

@kirill-konshin we typically develop INSIDE the monorepo. once there's something usable, i'd be happy to figure out a way to publish it on some special dist tag so you can dogfood it in your org.

@kirill-konshin
Copy link

@shilman sure, but I thought it's already in "shippable" state :)

@ndelangen
Copy link
Member Author

This branch is not in a shippable state.

@shilman shilman mentioned this pull request Sep 9, 2020
@stale stale bot added the inactive label Oct 4, 2020
@stale stale bot removed the inactive label Oct 5, 2020
@kirill-konshin
Copy link

Antibot.

@dudiharush
Copy link

Hi @kirill-konshin,
are you still working on the upgrade to support Webpack 5?

@storybookjs storybookjs deleted a comment from stale bot Oct 19, 2020
@kirill-konshin
Copy link

@dudiharush I've put it on hold due to too many errors. I've updated things here and there and got more and more errors, so I just stopped at some point.

@dudiharush
Copy link

Ok, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants